Skip to content

[Fix] Align OpenAI vllm_xargs value types across request schemas#48252

Open
sagearc wants to merge 9 commits into
vllm-project:mainfrom
sagearc:align-completion-request-xargs
Open

[Fix] Align OpenAI vllm_xargs value types across request schemas#48252
sagearc wants to merge 9 commits into
vllm-project:mainfrom
sagearc:align-completion-request-xargs

Conversation

@sagearc

@sagearc sagearc commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Some OpenAI request schemas accepted list-valued vllm_xargs while completion requests only accepted scalar values. This aligns the shared vllm_xargs type across completions, chat completions, and responses.

split out from #47922

Signed-off-by: Sage Ahrac <sagiahrak@gmail.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@mergify mergify Bot added the frontend label Jul 10, 2026
sagearc added 7 commits July 10, 2026 17:27
Signed-off-by: Sage <80211083+sagearc@users.noreply.github.com>
Signed-off-by: Sage <80211083+sagearc@users.noreply.github.com>
Removed type aliases for VLLMXArgsValue and VLLMXArgs.

Signed-off-by: Sage <80211083+sagearc@users.noreply.github.com>
Signed-off-by: Sage <80211083+sagearc@users.noreply.github.com>
@mergify

mergify Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @sagearc.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify Bot added the needs-rebase label Jul 12, 2026
Signed-off-by: Sage Ahrac <sagiahrak@gmail.com>
@mergify mergify Bot removed the needs-rebase label Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant